-
-
Notifications
You must be signed in to change notification settings - Fork 14.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cynthion: init at 0.1.7 #353392
cynthion: init at 0.1.7 #353392
Conversation
ff4bf2a
to
b85b818
Compare
175fdfa
to
7849b81
Compare
1806bf8
to
931a394
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In addition to the question in the line comments, I'm not sure what would be the required way to merge all of these dependency packages. Should they be separate PRs; or separate commits in this single PR; or it's fine to keep this PR single-commit (which was the least effort)
Guidance here would be appreciated!
Please separate commits in this one PR. Thanks. |
931a394
to
bcf6ed7
Compare
@natsukium I made the changes, please have a look when you can! ~Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks mostly good.
I left some comments, but they are trivial.
bcf6ed7
to
57fb63c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks. I'll merge when ofborg-eval is finished.
|
Python package and utilities for the Great Scott Gadgets Cynthion USB Test Instrument https://github.com/greatscottgadgets/cynthion
Packages (dependencies) also included in this PR:
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.